-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PhoneAuthProvider improvement for ios #528
Conversation
…brary, so there is no point to use it in PhoneVerificationProvider
I could also update this PR to just have FIRAuthUIDelegateProtocol as optional and have less changes in code: Instead of: That UIDelegate is mostly for viewcontroller based architecture and most of people use SwiftUI nowadays. |
Im not a maintainer or anything on this project but isnt is better to add an interface that has the same functions the delegate has? Then you can use that type for the delegate variable and can create a wrapper class for the custom delegate class here that will automatically call the underlying firebase delegate itself. Just sharing my idea here, this may turn out to be harder than i explain so take it with a grain of salt :) |
|
@BasBuijsen - thank you, actually ended up doing similar way as you described for now. |
Can Some one guide me on how to use this Phone Authentication i am able to send the otp but how to do verification |
FIRAuthUIDelegateProtocol is not available outside of the firebase-kotlin-sdk library, so there is no way to inject PhoneVerificationProvider to verifyPhoneNumber method. That makes PhoneAuthProvider impossible to use on IOS platform.